Repository navigation
fix: keep postponing when a task cannot be saved - #87
Merged
Merged
Conversation
During panic dismissal a "423 Locked" from Xandikos (its git index was locked by another write) aborted the whole run with a traceback. A PutError on saving a postponed task is now reported, and the rest of the list is still postponed. The task is not marked as postponed. If the parent a child needs moved cannot be saved, the child is reported as not postponed. A parent saved before its child fails stays moved. Prompt: (traceback from `plann interactive manage-tasks` pasted, ending in "caldav.lib.error.PutError: PutError at '423 Locked'" from _procrastinate's x.save()) Followup-Prompt: I think we have a github issue and some design document on "retry logic in the caldav library". Please find and add the 423-retry into the same design document. Fix the try-and-continue in plann. [the design document is in the caldav repository] Assisted-By: Claude Opus 5.5 <noreply@anthropic.com> Reviewed-by: Tobias Brox <tobias@redpill-linpro.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Retrying the 423 itself is left to caldav: see
docs/design/RETRY_AND_RESILIENCE_DESIGN.mdthere, tracked in python-caldav/caldav#695.Commit message: